Skip to content

test: Opensearch - add unit tests#3182

Merged
anakin87 merged 2 commits intomainfrom
opensearch-tests
Apr 20, 2026
Merged

test: Opensearch - add unit tests#3182
anakin87 merged 2 commits intomainfrom
opensearch-tests

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 20, 2026

Related Issues

Proposed Changes:

  • add a few unit tests
  • refactor conftest fixtures to reduce duplication
  • introduce utility method to avoid duplication

How did you test it?

CI

Checklist

@github-actions github-actions bot added integration:opensearch type:documentation Improvements or additions to documentation labels Apr 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 20, 2026

Coverage report (opensearch)

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  integrations/opensearch/src/haystack_integrations/components/retrievers/opensearch
  bm25_retriever.py
  embedding_retriever.py
  metadata_retriever.py
  open_search_hybrid_retriever.py
  sql_retriever.py 113-115, 162-164
  utils.py
  integrations/opensearch/src/haystack_integrations/document_stores/opensearch
  auth.py
Project Total  

This report was generated by python-coverage-comment-action

@anakin87 anakin87 marked this pull request as ready for review April 20, 2026 10:12
@anakin87 anakin87 requested a review from a team as a code owner April 20, 2026 10:12
@anakin87 anakin87 requested review from bogdankostic and removed request for a team April 20, 2026 10:12
@anakin87 anakin87 self-assigned this Apr 20, 2026
@anakin87 anakin87 merged commit 57fa671 into main Apr 20, 2026
9 checks passed
@anakin87 anakin87 deleted the opensearch-tests branch April 20, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:opensearch type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants